6 ajout dun objet prix sur chaque produit #10
Conversation
✅ Deploy Preview for matta-nuu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Super merci pour ces avancées !!
J'ai fait quelques commentaires ;)
Concernant l'utilisation de l'historique des prix, je pense que ça sera utilisé le jour ou on fera des stats du genre "sur l'année dis moi le prix total de chaque produit". Mais ça faut voir avec les utilisateurs quels sont leur besoins réels.
La première utilisation qu'on pourra faire avec les prix c'est calculer le cout d'une recette par personne, et peut etre l'afficher dans la liste des recettes, comme ça tu peux voir quelles sont les plus chères
|
Top ! Merci pour les fix :) Je pense que tu pourrais maintenant rajouter deux use case simple d'utilisation des prix Ordersrajouter dans les "Order" le prix total de la commande en utilisant les prix? (uniquement si au moins un des produits de la commande à un prix configuré, i.e. si le total est > 0) RecipieCalculer le prix par personne d'une recette. On pourrait afficher ce prix dans la liste des recettes avec une nouvelle colonne? Avec le meme warning si tous les produits n'ont pas de prix |
|
Je crois qu j'ai fait une erreur : j'ai fait un git rebase, et maintenant je crois que les commit que j'ai fait veulent être merge avec master. Mais il me reste encore à calculer le prix par personne d'une recette. |
Pour moi cela fonctionne. Peux-tu me préciser où cela ne fonctionne paspour toi ? |
|
Salut ! Maintenant, avec le changement sur la computed property, ça fonctionne ! Est ce que tu as d'autres choses que tu voulais faire dans cette PR? ou elle est prête selon toi? |
|
J'ai ajouté les prix dans la liste des recettes, ainsi que dans le détail des recettes. Une fois les bugs éventuels corrigés, ou bien si tu veux que je modifie des choses, pour moi la PR sera terminée. |
|
Ok cool ! @ousabd @Vichearith vous voulez bien tester les nouvelles features de Alexis sur les prix? Vous pouvez tester à cette adresse Ca utilise la meme DB que en prod, donc soit utilisez le compte de Dhamma Mahi avec prudence, soit utilisez [email protected] / behappy ou [email protected] / behappy |
|
Je vois avec JF s'il peut trouver quelqu'un qui connait bien Matta Nuu.
*Eric ROS*
Le jeu. 21 nov. 2024 à 08:55, Sebastian Castro ***@***.***> a
écrit :
… Ok cool ! @ousabd <https://github.com/ousabd> @Vichearith
<https://github.com/Vichearith> vous voulez bien tester les nouvelles
features de Alexis sur les prix?
Vous pouvez tester à cette adresse
https://deploy-preview-10--matta-nuu.netlify.app/#/
Ca utilise la meme DB que en prod, donc soit utilisez le compte de Dhamma
Mahi avec prudence, soit utilisez ***@***.*** / behappy ou
***@***.*** / behappy
—
Reply to this email directly, view it on GitHub
<#10 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADVMFBWTYPBYEFIH6SLPLOT2BWGXRAVCNFSM6AAAAABNECTWVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJQGMYDOMBVGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
|
Salut le Dhamma dev crew, je suis en ce moment à Dhamma Mahi en long terme à la cuisine donc si besoin de tester des fonctionnalités sur place je peux m'en occuper ! |
|
Nathalie qui est long terme au centre, et à l'intendance, s'occupe de tester. Elle m'a déja à fait des retours, j'ai quelques trucs a modifier. |
Ah super. Je t'appelle bientôt. |
|
@seballot Pour moi la PR est prête maintenant. |
Je tente la PR en mode draft.
J'ai normalement corrigé l'erreur que tu avais Sebastian, ainsi que tes suggestions.
Dis-moi si c'est mieux, et ce que l'on fait avec les dates dans l'historique des prix.